Skip to content

build: dedupe targets in api golden script#32669

Merged
crisbeto merged 1 commit intoangular:mainfrom
crisbeto:api-golden-dedupe
Jan 20, 2026
Merged

build: dedupe targets in api golden script#32669
crisbeto merged 1 commit intoangular:mainfrom
crisbeto:api-golden-dedupe

Conversation

@crisbeto
Copy link
Copy Markdown
Member

In the past we had one API golden target per component, but now the target is per package. This means that if we run something like approve-goldens material/button material/dialog material/datepicker, we'll run the same command three times.

These changes dedupe the set of targets.

In the past we had one API golden target per component, but now the target is per package. This means that if we run something like `approve-goldens material/button material/dialog material/datepicker`, we'll run the same command three times.

These changes dedupe the set of targets.
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Jan 18, 2026
@pullapprove pullapprove Bot requested a review from josephperrott January 18, 2026 09:31
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label Jan 18, 2026
Copy link
Copy Markdown
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Jan 20, 2026
@crisbeto crisbeto merged commit 1f8b4b5 into angular:main Jan 20, 2026
29 of 31 checks passed
@crisbeto
Copy link
Copy Markdown
Member Author

This PR was merged into the repository. The changes were merged into the following branches:

crisbeto added a commit that referenced this pull request Jan 20, 2026
In the past we had one API golden target per component, but now the target is per package. This means that if we run something like `approve-goldens material/button material/dialog material/datepicker`, we'll run the same command three times.

These changes dedupe the set of targets.

(cherry picked from commit 1f8b4b5)
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Feb 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants